response.js ➔ trim - Code Metrics - UnitApi/js - Measure and Improve Code Quality continuously with Scrutinizer

response.js ➔ trim A
last analyzed 2020-02-10 08:27 UTC

↳ Parent: response.js

Complexity

Conditions

Size

Total Lines	3
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	2
dl	0
loc	3
rs	10
c	0
b	0
f	0
cc	1

function trim(str) {
    return str.replace(/^\s+|\s+$/g, '');
}


function isJson(str) {
    try {
        JSON.parse(str);
    } catch (e) {
        return false;
    }
    return true;
}

function Response(xhr, error, success) {
    if (!isJson(xhr.responseText)) {
        console.error('Response give not JSON Data');
        // alert('Response give not JSON Data');
        console.log(xhr.responseText);

        return false;
    }
    var data = JSON.parse(xhr.responseText);
    if (xhr.readyState == 4 && xhr.status == "200") {
        // console.table(data);
        success(data, xhr);

    } else {
        // console.error(data);
        error(data, xhr);

    }
};


1			function trim(str) {
2			return str.replace(/^\s+\|\s+$/g, '');
3			}
4
5
6			function isJson(str) {
7			try {
8			JSON.parse(str);
9			} catch (e) {
10			return false;
11			}
12			return true;
13			}
14
15			function Response(xhr, error, success) {
16			if (!isJson(xhr.responseText)) {
17			console.error('Response give not JSON Data');
18			// alert('Response give not JSON Data');
19			console.log(xhr.responseText);
			0 ignored issues – show Debugging Code introduced 2019-10-09 13:01 UTC by Report Bug Copy Issue Report `console.log` looks like debug code. Are you sure you do not want to remove it? Loading history...
20			return false;
21			}
22			var data = JSON.parse(xhr.responseText);
23			if (xhr.readyState == 4 && xhr.status == "200") {
24			// console.table(data);
25			success(data, xhr);
			0 ignored issues – show Best Practice introduced 2019-10-09 13:01 UTC by Report Bug Copy Issue Report There is no `return` statement in this branch, but you do return something in other branches. Did you maybe miss it? If you do not want to return anything, consider adding `return undefined;` explicitly. Loading history...
26			} else {
27			// console.error(data);
28			error(data, xhr);
			0 ignored issues – show Best Practice introduced 2019-10-09 13:01 UTC by Report Bug Copy Issue Report There is no `return` statement in this branch, but you do return something in other branches. Did you maybe miss it? If you do not want to return anything, consider adding `return undefined;` explicitly. Loading history...
29			}
30			};
31

UnitApi / js

response.js ➔ trim A last analyzed 2020-02-10 08:27 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

response.js ➔ trim A
last analyzed 2020-02-10 08:27 UTC